Skip to content

Conversation

@chillenzer
Copy link
Contributor

Fixes a deadlock popping up applying the test suite for FlatterScatter to Scatter.

@chillenzer chillenzer marked this pull request as draft August 23, 2024 12:36
@chillenzer
Copy link
Contributor Author

Offline discussion: This is not so much a bug as a misimplementation of the resetfreedpages feature because it simply means that pages are potentially left with a chunk size while no thread holds memory there. While it can have unfortunate implications concerning memory availability (which led to deadlocking in the stress test code, i.e. user code), it is not an urgent bug in the strict sense or a deadlock in the library code. Setting this to draft as a reminder that we might want to do a major refactoring of the algorithm in order to establish an invariant that can be used to prove correctness then.

@chillenzer chillenzer added the bug label Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant